Skip to content

Surface live worktree agents in the VS Code companion#301

Merged
NagyVikt merged 1 commit intomainfrom
agent/codex/vscode-active-agents-live-worktree-telem-2026-04-22-13-43
Apr 22, 2026
Merged

Surface live worktree agents in the VS Code companion#301
NagyVikt merged 1 commit intomainfrom
agent/codex/vscode-active-agents-live-worktree-telem-2026-04-22-13-43

Conversation

@NagyVikt
Copy link
Copy Markdown
Collaborator

Automated by gx branch finish (PR flow).

The extension now falls back to root-level AGENT.lock telemetry when no launcher session JSON exists, merges those synthetic rows with normal active-session records, and watches AGENT.lock changes so SCM stays live for direct gx worktree lanes.

Constraint: Launcher session JSON is optional for direct gx branch start lanes
Rejected: Require every live lane to run through codex-agent wrapper | misses existing worktree telemetry and keeps SCM blind
Confidence: high
Scope-risk: moderate
Directive: Keep AGENT.lock fallback secondary to launcher-backed session rows and never duplicate the same worktree in SCM
Tested: node --test test/vscode-active-agents-session-state.test.js
Tested: openspec validate agent-codex-vscode-active-agents-live-worktree-telem-2026-04-22-13-43 --type change --strict
Tested: openspec validate --specs
Not-tested: Manual VS Code window verification against a live workspace
@NagyVikt NagyVikt merged commit 57947e3 into main Apr 22, 2026
@NagyVikt NagyVikt deleted the agent/codex/vscode-active-agents-live-worktree-telem-2026-04-22-13-43 branch April 22, 2026 11:55
NagyVikt added a commit that referenced this pull request Apr 22, 2026
The feature branch already merged through PR #301, but the change artifact still showed cleanup unchecked because the finish flow completed after the commit. This follow-up captures the actual PR, merge commit, and post-finish prune evidence so the OpenSpec record matches reality.

Constraint: Cleanup evidence only existed after the original branch had already merged and deleted its refs
Rejected: Leave the merged tasks file stale | violates the repo cleanup-evidence contract and hides the real PR result
Confidence: high
Scope-risk: narrow
Directive: When Guardex finish merges after the last feature commit, record the resulting PR and prune evidence in a tiny follow-up lane instead of pretending the artifact is already truthful
Tested: git diff --check
Tested: git worktree list --porcelain
Tested: git branch -a --list agent/codex/vscode-active-agents-live-worktree-telem-2026-04-22-13-43 remotes/origin/agent/codex/vscode-active-agents-live-worktree-telem-2026-04-22-13-43

Co-authored-by: NagyVikt <nagy.viktordp@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant